Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support for a custom apt source release #1561

Merged
merged 1 commit into from
Jan 3, 2024

Conversation

h0tw1r3
Copy link
Contributor

@h0tw1r3 h0tw1r3 commented Dec 19, 2023

Postgresql no longer specifically supports Ubuntu 18.04, but still supports Debian 10 which can be used until Puppet drops support for Ubuntu 18.04.

Summary

Postgresql recommends using the Debian buster repo for Ubuntu bionic.

Additional Context

Temporary until Puppet no longer supports Ubuntu 18.04.

Related Issues (if any)

voxpupuli/puppet-puppetboard#402

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.

ekohl
ekohl previously requested changes Dec 23, 2023
Copy link
Collaborator

@ekohl ekohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMHO we shouldn't do this and instead drop 18.04. The standard support of 18.04 has ended and clearly PostgreSQL no longer wants to supports it since they dropped it from their repositories (on 2023-11-05 according to https://wiki.postgresql.org/wiki/Apt#News).

If anything, we can introduce a parameter to apt_postgresql_org for the release so you can override it via Hiera. In fact, I think that'd be wise for everything including the GPG keys.

Useful in cases where postgresql has archived support for the platform.
@h0tw1r3
Copy link
Contributor Author

h0tw1r3 commented Dec 23, 2023

If anything, we can introduce a parameter to apt_postgresql_org for the release so you can override it via Hiera. In fact, I think that'd be wise for everything including the GPG keys.

Reworked. Thanks for the constructive feedback. I think the overall design pattern could use attention which is why I'm not touching key management yet.

@h0tw1r3 h0tw1r3 changed the title fix ubuntu 18.04 invalid repo support for a custom apt source release Dec 23, 2023
Copy link
Collaborator

@ekohl ekohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't properly review this right now, but at first glance this looks like it addresses my concerns.

Comment on lines +2 to +3
postgresql::repo::baseurl: https://apt-archive.postgresql.org/pub/repos/apt/
postgresql::repo::release: "%{facts.os.distro.codename}-pgdg-archive"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Having archive repos is much cleaner than using a different distro. I much prefer this.

@ekohl ekohl dismissed their stale review December 24, 2023 03:54

Concerns addressed

Copy link
Collaborator

@ekohl ekohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the overall design pattern could use attention which is why I'm not touching key management yet.

I think #1563 would be a major step in that direction. If you just need to provide a URL without an ID then it's way easier.

@ekohl ekohl merged commit 010ed69 into puppetlabs:main Jan 3, 2024
34 of 42 checks passed
@h0tw1r3 h0tw1r3 deleted the fix-ubuntu-18-repo branch January 8, 2024 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants